Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

APC single crew #26

Merged
merged 5 commits into from
Oct 27, 2023
Merged

APC single crew #26

merged 5 commits into from
Oct 27, 2023

Conversation

Doubleumc
Copy link
Contributor

About the pull request

Allows the APC to be run by a single crewman by transferring weapons control from the gunner to the driver.

Transferred the weapon control verbs from the (primary) gunner to the driver, and set the hardpoints to listen to the driver.

In testing driver can still drive, honk the horn, lock the doors, and they can now swap between and fire primary, secondary, and support hardpoints. Transport variant side gunners can still use their side guns. Gunner seat is effectively dummied out, its basically an observer seat now.

This is probably a naïve way to do it, but it works and touches minimal code.

Explain why it's good for the game

Multicrew aspect of APCs seemed undesired, single crew is now possible.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
qol: APC weapons control transferred to driver
/:cl:

Switch APC weapons control from gunner to driver.
Forgot to change the verbs for the medical and command variants.
Now only APC hardpoints are controlled by driver, tank is untouched and still functional. No longer grant `Toggle Shift Click` to drivers as that verb is hard-coded for gunners and changing that is out of scope.
@morrowwolf morrowwolf merged commit ece3f43 into cmss13-devs:master Oct 27, 2023
26 checks passed
@Doubleumc Doubleumc deleted the apc branch October 27, 2023 05:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants